Skip to content

feat(ai): migrate ChatGPT authentication and inference - #291

Merged
Pepps233 merged 10 commits into
fix/remove-radius-integrationfrom
feat/chatgpt-public-auth
Oct 27, 2026
Merged

Pepps233 merged 10 commits into
fix/remove-radius-integrationfrom
feat/chatgpt-public-auth

Conversation

@Pepps233

@Pepps233 Pepps233 commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Register Diffex ChatGPT sessions and use account-specific models through the public Responses API.

Reasoning

Authentication, granted plan usage, model discovery, inference, and sign-out must share the same issued registration.
Transient failures should preserve usable credentials; terminal refresh failures require explicit sign-in.

Changes

  • Persist stable host identities and issued registrations; validate sign-in tokens and consent.
  • Support saved registration selection and remote callback completion.
  • Fetch account-specific catalogs with isolated caches and report subscription usage.
  • Use public streaming Responses requests with storage disabled.
  • Coordinate cancellation, credential writes, token expiry, and logout revocation.
  • Share credential reloads across active storage paths without retaining inactive credentials.
  • Use private test directories, owner-only credential writes, and persistent example storage under the user's home.
  • Pin the token-validation dependency and refresh associated lockfiles.

Testing

  • npm run check and ./test.sh passed on the complete local revision.
  • Controlled OAuth, registration, model-cache, inference, credential synchronization, and logout tests passed.
  • Registration path traversal and credential permission regressions passed.
  • Validation used synthetic credentials and mocked provider responses.

Persist issued client and host identities, validate sign-in tokens, and revoke renewable sessions on logout.
Discover account-specific models and stream public Responses requests with isolated catalog caches and subscription usage reporting.
Retain validated registrations across consent and terminal refresh failures, protect credential writes, and keep cancellation and identity hints out of error output.
@Pepps233 Pepps233 added enhancement New feature or request coding-agent labels Oct 27, 2026
@Pepps233 Pepps233 self-assigned this Oct 27, 2026
@Pepps233
Pepps233 added this pull request to stack #293 October 27, 2026 13:31
Comment thread packages/coding-agent/test/auth-storage.test.ts Fixed
Comment thread packages/coding-agent/test/auth-storage.test.ts Fixed
Comment thread packages/coding-agent/src/core/auth-storage.ts Dismissed
Comment thread packages/coding-agent/src/core/auth-storage.ts Dismissed
Comment thread packages/coding-agent/src/core/auth-storage.ts Dismissed
Comment thread packages/ai/src/auth/oauth/chatgpt-registration.ts Dismissed
Comment thread packages/ai/src/auth/oauth/chatgpt-registration.ts Dismissed
Comment thread packages/ai/src/auth/oauth/chatgpt-registration.ts Dismissed
Comment thread packages/ai/src/auth/oauth/chatgpt-registration.ts Dismissed
Comment thread packages/ai/src/auth/oauth/chatgpt-registration.ts Dismissed
- Check the public account display name so native artifact acceptance completes after authentication migration
- Keep callback and login checks consistent with the public account integration
- Use private temporary directories and inspect one file descriptor

- Share credential reloads for each active storage path without retaining inactive credentials
- Keep persistent example credentials outside shared temporary directories

- Create private compaction fixtures and inspect credential metadata through file descriptors
- Keep exclusive credential writes visibly owner-only to security analysis
Comment thread packages/coding-agent/test/auth-storage.test.ts Dismissed
Comment thread packages/coding-agent/src/core/auth-storage.ts Dismissed
@Pepps233
Pepps233 merged commit 45bfc2e into main Oct 27, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

coding-agent enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants